Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Correction du paramétrage des ressources à utiliser avec le SearchEngine (Ol + Leaflet) #377

Merged
merged 2 commits into from
May 28, 2024
Merged

Correction du paramétrage des ressources à utiliser avec le SearchEngine (Ol + Leaflet) #377

merged 2 commits into from
May 28, 2024

Conversation

elias75015
Copy link
Contributor

@elias75015 elias75015 commented May 16, 2024 •
edited
Loading

Pull request checklist

Verifiez que votre Pull Request remplit les conditions suivantes :

  • Des tests ont été ajoutés pour les changements (corrections de bugs ou features)
  • De la documentation a été mise à jour ou ajoutée si nécessaire (corrections de bugs ou features)
  • Un build (npm run build) a été lancé localement et s'est correctement déroulé
  • Les exemples impactés par les modifications (npm run samples) ont été testés et validés localement
  • Les tests (npm run test) sont passés localement

Type de Pull request

Quel type de changement cette Pull Request introduit-elle :

  • Bugfix
  • Feature
  • Mise à jour du style du code (syntaxe, renommage de fonctions)
  • Refactoring (lisibilité/performance du code, sans changements fonctionnels)
  • Changement sur le processus de build
  • Contenu de la documentation
  • Autres (décrire ci-après) :

Quel est le comportement actuel (avant PR) :

Imposible de paramétrer les ressources à utiliser sur le SearchEngine (limité les entrées de la listes déroulantes selon son paramétrage)

Quel est le nouveau comportement :

Les options par défaut sont prises en compte, et le paramétre resources.geocode est bien pris en compte sur les extensions OpenLayer et Leaflet.

Exemple d'utilisation avec extension Géoplateforme pour Leaflet :

var search = L.geoportalControl.SearchEngine({
              collapsed : false,
              displayInfo : true,
              placeholder : "Recherche de points d'interêt...",
              displayMarker : true,
              markerStyle : "green",
              zoomTo : "auto",
              resources : {
                geocode: ["CadastralParcel"]
              },
              position : "bottomright",
              displayAdvancedSearch : true,
              advancedSearch : {}
            });

--> Seules les parcelles cadastrales peuvent être recherchée en recherche avancée.

Exemple d'utilisation avec extension Géoplateforme pour OpenLayers :

                    var search = new ol.control.SearchEngine({
                        collapsed : false,
                        displayAdvancedSearch : true,
                        advancedSearch : {
                          PositionOfInterest : [
                            {name : "nature", title : "Nature"},
                            {name : "municipality", title : "Ville"},
                            {name : "department", title : "Departement", value : "77"}
                          ],
                          // on ne veut pas configurer cette entrée qui est donc présente mais inactive !
                          StreetAddress : null || [], 

                        },
                        resources : {
                          geocode : ["CadastralParcel", "StreetAddress", "PositionOfInterest"],
                          autocomplete : ["PositionOfInterest"]
                          // autocomplete : ["StreetAddress", "test"]
                        },
                        geocodeOptions : {
                          serviceOptions : {
                            filterOptions : {
                              type : ["PositionOfInterest"],
                              // department : "31"
                            },
                            maximumResponses : 5,
                            returnFreeForm : false,
                            // rawResponse : true
                          }    
                        },
                        autocompleteOptions : {
                          filterOptions : {
                            type : ["StreetAddress", "PositionOfInterest"],
                          }
                        },
                        // valeur possible : "" ou null, "auto", 18, function (i) { return 1; }
                        zoomTo : "auto",
                        placeholder : "Recherche...",
                        markerDisplay : true,
                        markerStyle : "turquoiseBlue"
                    });

--> On recherche sur les trois index possible, mais en recherche avancée, le paramétrage interdit la recherche sur StreetAdress

Cette PR introduit-elle des breaking changes ?

  • Oui
  • Non

Autres informations

Attention à bien vérifié que rien ne casse dans les briques en aval.
Si validées, modifications à reporter sur le projet extensions-openlayers.

elias75015 added 2 commits May 16, 2024 19:27
@elias75015
fix(Controls): Paramétrage des ressources de la recherche avancée cor…
b29a2a3 
…rigée

- fonction mergeParams avec test d'existece trop sévère corrigé
- nom des ressources à utilisé pas bob sous openlayer (poi,address au lieu de PositionOfInterest, StreetAddress)
@elias75015
style(eslint): fix eslint errors
223bf42
@elias75015 elias75015 self-assigned this May 16, 2024
@elias75015 elias75015 requested a review from lowzonenose May 16, 2024 17:38
@elias75015 elias75015 merged commit 01ba4d2 into develop May 28, 2024
4 checks passed
@elias75015 elias75015 deleted the fix/param-resources-searchengine branch May 28, 2024 12:48
@elias75015 elias75015 mentioned this pull request May 28, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lowzonenose lowzonenose Awaiting requested review from lowzonenose

Assignees

@elias75015 elias75015

Labels
🐛 bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@elias75015